Skip to content

Update Resource Groups docs for the redesigned UI - #2751

Merged
gary149 merged 7 commits into
mainfrom
resource-groups-ui-refresh
Sep 15, 2026
Merged

gary149 merged 7 commits into
mainfrom
resource-groups-ui-refresh

Conversation

@gary149

@gary149 gary149 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor
image

The Resource Groups pages in org settings were redesigned recently, this updates the docs to match.

Changes

  • Getting started now describes the resource group page and its four tabs: Overview (summary chips and resource/user previews), Resources, Users, and Settings.
  • Auto-join and Spend limits instructions now point at their new home in the group Settings tab, and the auto-join section gained a screenshot.
  • All screenshots retaken for the new UI, in light and dark mode at 3x, using a demo org (no real data). The images are already uploaded to the documentation-images dataset under their existing names, so this PR only touches the markdown.
  • Accuracy fixes, verified against the actual behavior:
    • "Read+" creation setting excludes members with the no_access org role, the previous wording said any org member.
    • Granular feature access is enforced as a membership-based permission gate org-wide, not as a requirement that resources live inside the allowed groups.
    • Jobs cost attribution uses the resourceGroupId field in the job payload, not the namespace.
  • Copy cleanup: key terms bolded, nested bullets flattened, role names shown as code, creator role labels matched to the UI (Write+, Contributor+).

The enterprise-resource-groups page gets updated screenshot alt text.


Note

Low Risk
Documentation-only changes with no runtime or security impact; accuracy fixes reduce misconfiguration risk for admins.

Overview
Updates Team & Enterprise Resource Groups documentation to match the redesigned org settings UI—markdown only; screenshots were re-uploaded to the documentation-images dataset under new hub asset names.

Getting started now walks through the group page’s four tabs (Overview, Resources, Users, Settings), retargets screenshots (overview, add-resources modal), drops the old “move repo” image, and adds descriptive alt text across pages including enterprise-resource-groups.md.

Auto-join and Spend limits instructions point at the Settings tab (Auto-include org members, spend limits card) with new screenshots; auto-join scope is reframed around the Include no_access members checkbox.

Behavioral corrections: Read+ creation excludes no_access org members; granular feature access is documented as a who gate org-wide, not a requirement that resources live in allowed groups; Jobs billing uses resource_group_id (plus org namespace), not namespace alone.

Copy edits: bold key terms, code role names, UI-aligned labels (Write+, Contributor+), and clearer Resources/Members bullets.

Reviewed by Cursor Bugbot for commit b0cca85. Bugbot is set up for automated code reviews on this repo. Configure here.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-15T08:40:34.145603Z b0cca85 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e4ed6491f5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/hub/security-resource-groups.md Outdated
Comment thread docs/hub/security-resource-groups.md Outdated
Comment thread docs/hub/security-resource-groups.md Outdated
Comment thread docs/hub/security-resource-groups.md Outdated
gary149 and others added 3 commits September 15, 2026 10:34
Co-authored-by: Julien Chaumond <julien@huggingface.co>
Co-authored-by: Daniel van Strien <davanstrien@users.noreply.github.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b0cca8597f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


- **All org members**: Include every member, including members with the `no_access` organization role.
- **Read+ members only**: Exclude members with the `no_access` organization role.
- **All org members** (default): leave **Include no_access members** unchecked to exclude members with the `no_access` organization role.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Correct the mislabeled default auto-join scope

The unchecked state is labeled All org members even though this sentence says it excludes no_access members, and the following paragraph calls that same scope Read+ members only. This makes it unclear which setting actually enrolls every organization member; label the unchecked/default scope Read+ members only and reserve All org members for the checked state.

Useful? React with 👍 / 👎.

@gary149
gary149 merged commit 65ee22d into main Sep 15, 2026
3 checks passed
@gary149
gary149 deleted the resource-groups-ui-refresh branch September 15, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants